Skip to content

Comments

Fix: rename requirements#379

Merged
S1ro1 merged 1 commit intomainfrom
fix-requirements
Dec 14, 2025
Merged

Fix: rename requirements#379
S1ro1 merged 1 commit intomainfrom
fix-requirements

Conversation

@S1ro1
Copy link
Member

@S1ro1 S1ro1 commented Dec 14, 2025

No description provided.

Copilot AI review requested due to automatic review settings December 14, 2025 23:20
@S1ro1 S1ro1 merged commit 2e77e4a into main Dec 14, 2025
5 of 9 checks passed
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR renames the requirements file from requirements.txt to requirements-dev.txt and updates references throughout the codebase to use the new filename. This appears to be a refactoring to better distinguish development dependencies from potential production dependencies.

  • Adds new requirements-dev.txt file with 15 package dependencies
  • Updates installation instructions in README.md to reference the new filename
  • Updates the NVIDIA workflow to install from the renamed file

Reviewed changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated no comments.

File Description
requirements-dev.txt New requirements file containing all development dependencies
README.md Updated installation instructions to reference requirements-dev.txt
.github/workflows/nvidia_workflow.yml Updated pip install command to use requirements-dev.txt
docker/amd-docker.Dockerfile Formatting-only change (no functional impact)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant